Skip to content

Conversation

pcwalton
Copy link
Contributor

@pcwalton pcwalton commented May 9, 2013

@catamorphism
Copy link
Contributor

Ok except for the documentation, as per my line comments -- I'll r+ after that's fixed.

@pcwalton pcwalton closed this May 23, 2013
@pcwalton pcwalton deleted the de-extern-mod branch May 23, 2013 05:05
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 2, 2021
…=flip1995

Reassign default private

changelog: fix field_reassign_with_default false positive

* Fix rust-lang#6344
* Fix assumption that `field: Default::default()` is the same as `..Default::default()`
* Cleanup some redundant logic
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 4, 2021
[beta] Backport of rust-lang#6375 - field_reassign_with_default fix

With the pinned nightly we can test backports to our beta branch now 🎉

cc rust-lang#6515

changelog: beta 1.50: Backport of private fields fix in [`field_reassign_with_default`] lints
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants